b1d47ff224b00f4437b7d052d29d63722bada029,clc/modules/euare/src/main/java/com/eucalyptus/auth/euare/EuareService.java,EuareService,updateGroup,#UpdateGroupType#,717

Before Change


      if ( request.getNewGroupName( ) != null && !"".equals( request.getNewGroupName( ) ) ) {
        groupFound.setName( request.getNewGroupName( ) );
      }
      if ( request.getNewPath( ) != null && !"".equals( request.getNewPath( ) ) ) {
        groupFound.setPath( sanitizePath( request.getNewPath( ) ) );
      }
    } catch ( Exception e ) {

After Change


    User requestUser = ctx.getUser( );
    Account account = getRealAccount( ctx, request.getDelegateAccount( ) );
    Group groupFound = null;
    if ( Strings.isNullOrEmpty( request.getNewGroupName( ) ) ) {
      try {
        USER_AND_GROUP_NAME_CHECKER.check( request.getNewGroupName( ) );
      } catch ( InvalidValueException e ) {
        throw new EuareException( HttpResponseStatus.BAD_REQUEST, EuareException.INVALID_NAME, "New group name: " + request.getNewGroupName( ) + " is invalid." );
      }      
    }
    if ( Strings.isNullOrEmpty( request.getNewPath( ) ) ) {
      try {
        PATH_CHECKER.check( request.getNewPath( ) );
      } catch ( InvalidValueException e ) {
        throw new EuareException( HttpResponseStatus.BAD_REQUEST, EuareException.INVALID_NAME, "New path: " + request.getNewPath( ) + " is invalid." );
      }
    }
    try {